Skip to content

Conversation

@Andrewpini
Copy link
Contributor

@Andrewpini Andrewpini commented Sep 22, 2022

Applies the following alterations to CDB behaviour in mesh shell

  • Adds feature that deletes the shell CDB node after perfoming a config client reset of another node.
  • Adds feature that clears the CDB entry on local reset.

Signed-off-by: Anders Storrø [email protected]

Adds feature that deletes the shell CDB node after perfoming a config
client reset of another node.

Signed-off-by: Anders Storrø <[email protected]>
@Andrewpini Andrewpini requested a review from PavelVPV September 22, 2022 13:53
@Andrewpini Andrewpini requested a review from nashif as a code owner September 26, 2022 07:44
@Andrewpini Andrewpini force-pushed the shell_cdb_del_node_on_reset branch from 095a423 to 77efbfe Compare September 26, 2022 08:38
@Andrewpini Andrewpini changed the title Bluetooth: Mesh: Shell: Del CDB node on node reset Bluetooth: Mesh: Shell: Update CDB clear/del behaviour Sep 26, 2022
alxelax
alxelax previously approved these changes Sep 26, 2022
PavelVPV
PavelVPV previously approved these changes Sep 26, 2022
@PavelVPV PavelVPV added this to the v3.3.0 milestone Sep 26, 2022
jhedberg
jhedberg previously approved these changes Sep 27, 2022
Adds feature that clears the CDB entry on local reset.

Signed-off-by: Anders Storrø <[email protected]>
@Andrewpini Andrewpini dismissed stale reviews from jhedberg, PavelVPV, and alxelax via 8af0dec September 27, 2022 08:21
@Andrewpini Andrewpini force-pushed the shell_cdb_del_node_on_reset branch from 77efbfe to 8af0dec Compare September 27, 2022 08:21
@Andrewpini Andrewpini requested review from PavelVPV, alxelax, jhedberg and mia-ko and removed request for mia-ko September 27, 2022 08:22
@Andrewpini Andrewpini requested review from akredalen and mia-ko and removed request for mia-ko September 27, 2022 11:04
@fabiobaltieri fabiobaltieri merged commit b9f776d into zephyrproject-rtos:main Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants